Skip to content

Conversation

@silabs-JeevanRH
Copy link
Contributor

@silabs-JeevanRH silabs-JeevanRH commented Oct 25, 2025

Description

This PR is to setup CI to build and test machine_learning_applications repo apps

Fixes # MLSW-10552

Type of change

Please select all the options that apply. Ideally only one should be selected.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

The goal of this task was to verify that all machine learning applications in the repository build successfully whenever any push is made. Since the CI pipeline was newly created, it was tested by pushing the updated workflow code to the repository, triggering the GitHub Actions job, and confirming that all matrix builds executed successfully end-to-end without errors, ensuring the pipeline correctly generates and compiles every ML application automatically.

Test Configuration:

  • Firmware version: N/A
  • Hardware: BRD2601B
  • Toolchain: ARM GNU GCC 12.2.rel1
  • SDKs: Simplicity sdk v2025.6.2, AI/ML extension v2.1.2

Screenshots

GITHUB_ACTION_RESULT

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

Copy link
Contributor

@silabs-raashid silabs-raashid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some work is needed on the CI job, please address those comments and we should be good to go. Marking this as "Request Changes" for now.

Copy link
Contributor

@silabs-raashid silabs-raashid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few minor changes needed again. But we're close to the finish line.

@silabs-raashid
Copy link
Contributor

Also, please resolve the conflict that this file, people_flow_counter_mls90460.slcp

@silabs-JeevanRH silabs-JeevanRH force-pushed the MLSW-10552-Setup-CI-to-build-and-test-machine_learning_applications-repo branch from c01f337 to 48f6d4e Compare November 6, 2025 11:12
@silabs-JeevanRH silabs-JeevanRH requested a review from a team as a code owner November 6, 2025 11:12
Copy link
Contributor

@silabs-raashid silabs-raashid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@silabs-JeevanRH silabs-JeevanRH force-pushed the MLSW-10552-Setup-CI-to-build-and-test-machine_learning_applications-repo branch from 48f6d4e to cc5e167 Compare November 6, 2025 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants